Conversation
37ea81a to
9d54c8b
Compare
There was a problem hiding this comment.
Pull request overview
This PR tightens MCP server selection behavior to require an explicit matching server name (entrypoint) and updates dependencies to a newer uipath build intended to support entrypoint autodiscovery.
Changes:
- Removed the “single server fallback” behavior in
McpConfig.get_server()so lookup is strictly by name. - Updated dependency versions (notably
uipath,uipath-runtime,uipath-core) and lockfile entries; added UV source configuration for TestPyPI. - Ignored a root-level
.claude/directory in.gitignore.
Reviewed changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
uv.lock |
Updates locked versions and switches uipath resolution to TestPyPI dev build; adds uipath-platform dependency. |
src/uipath_mcp/_cli/_utils/_config.py |
Makes get_server(name) strictly return only the named server (no single-server fallback). |
pyproject.toml |
Pins uipath to a dev version and configures UV to pull it from TestPyPI. |
.gitignore |
Adds .claude/ at repo root to ignored paths. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
pyproject.toml
Outdated
| # "uipath>=2.8.23, <2.9.0", | ||
| "uipath==2.10.12.dev1014315279", |
There was a problem hiding this comment.
uipath is pinned to a TestPyPI-only dev build (2.10.12.dev1014315279). This will make pip install uipath-mcp fail for end users because pip won’t consult [tool.uv.*] indexes/sources and won’t find that version on PyPI. Please switch back to a stable PyPI release before merging, or (if this must remain temporarily) move this to a dev-only dependency path and/or use a PEP 508 direct URL dependency so non-uv installers can resolve it deterministically.
| # "uipath>=2.8.23, <2.9.0", | |
| "uipath==2.10.12.dev1014315279", | |
| "uipath>=2.8.23, <2.9.0", | |
| # "uipath==2.10.12.dev1014315279", |
09ae039 to
ea28f73
Compare
ea084be to
6e6afd4
Compare
6e6afd4 to
51539eb
Compare
get_server()so that server lookup now strictly uses the provided name instead of returning the only available server if only one existsuipathto enable entrypoints autodiscovery (related to feat: add entrypoint autodiscover uipath-python#1431)to do:
uipathdev version with a stable version